Skip to content

Conversation

@petyosi
Copy link
Collaborator

@petyosi petyosi commented Nov 14, 2025

No description provided.

@petyosi petyosi requested a review from Copilot November 14, 2025 11:42
Copilot finished reviewing on behalf of petyosi November 14, 2025 11:44
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR renames the core packages to improve clarity and naming conventions:

  • @pydantic/logfire-apilogfire (core API used across all runtimes)
  • logfire@pydantic/logfire-node (Node.js-specific SDK with auto-instrumentation)

Key Changes

  • Updated package names in package.json files across all packages
  • Updated all import statements from old to new package names
  • Updated documentation (README.md, CLAUDE.md) to reflect new naming
  • Created changeset files documenting the breaking changes

Reviewed Changes

Copilot reviewed 33 out of 47 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/logfire-api/package.json Renamed package to logfire
packages/logfire-node/package.json Renamed package to @pydantic/logfire-node, updated dependency from @pydantic/logfire-api to logfire
packages/logfire-node/src/*.ts Updated imports from @pydantic/logfire-api to logfire
packages/logfire-cf-workers/package.json Updated dependency to logfire
packages/logfire-cf-workers/src/*.ts Updated imports to logfire
packages/logfire-browser/package.json Updated dependency to logfire
packages/logfire-browser/src/index.ts Updated imports to logfire
examples/*/package.json Updated dependencies to use new package names
examples//src/.ts Updated imports to use new package names
README.md Updated installation and usage instructions
CLAUDE.md Updated documentation to reflect new package structure
.changeset/*.md Added breaking change documentation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@samuelcolvin samuelcolvin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@petyosi petyosi merged commit 28eb056 into main Nov 17, 2025
8 checks passed
@petyosi petyosi deleted the package-rename branch November 17, 2025 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants